Skip to content

$zcvt is not required#1

Open
eduard93 wants to merge 3 commits intonikitaeverywhere:masterfrom
objectscript:master
Open

$zcvt is not required#1
eduard93 wants to merge 3 commits intonikitaeverywhere:masterfrom
objectscript:master

Conversation

@eduard93
Copy link
Copy Markdown

@eduard93 eduard93 commented Jun 3, 2016

No description provided.

@nikitaeverywhere
Copy link
Copy Markdown
Owner

So this is what I got now:
2016-06-03_144416

Ensemble 2016.2.0.636.0 / Windows 10.

Are you using Linux? I suppose $zcvt is still required.

@eduard93
Copy link
Copy Markdown
Author

eduard93 commented Jun 3, 2016

win10 / Cache 16.1 / Unicode

@nikitaeverywhere
Copy link
Copy Markdown
Owner

nikitaeverywhere commented Jun 3, 2016

@eduard93, my is unicode, too.

Any ideas why this is not consistent between our machines?

@eduard93
Copy link
Copy Markdown
Author

eduard93 commented Jun 3, 2016

Try:
do req.SetHeader("Content-Type", "application/json; charset=UTF-8;")

@nikitaeverywhere
Copy link
Copy Markdown
Owner

This also doesn't help.

@gnibeda
Copy link
Copy Markdown

gnibeda commented Jun 3, 2016

tested on win7 cache 2016.1.0.617.0, in windows notepad encoding is ok.

@eduard93
Copy link
Copy Markdown
Author

eduard93 commented Jun 3, 2016

@gnibeda with or without $zcvt?

@nikitaeverywhere
Copy link
Copy Markdown
Owner

@eduard93, I asked him to try your version.

@nikitaeverywhere
Copy link
Copy Markdown
Owner

Here are the files:
chat (my).txt
chat (Anton's).txt

When I open Anton's file in Notepad, I got wrong encoding. However, when I open this file in Notepad++ it is OK.

@eduard93
Copy link
Copy Markdown
Author

eduard93 commented Jun 3, 2016

@nikitaeverywhere
Copy link
Copy Markdown
Owner

@eduard93, maybe refer this conversation on DC? And the two version of code.

Repository owner deleted a comment from hyassine1 Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants